Skip to content

test: wait-for-hydra SSE endpoint#241

Closed
angerman wants to merge 6 commits into
mainfrom
test/wait-for-hydra-sse
Closed

test: wait-for-hydra SSE endpoint#241
angerman wants to merge 6 commits into
mainfrom
test/wait-for-hydra-sse

Conversation

@angerman
Copy link
Copy Markdown
Collaborator

Summary

  • Test PR to validate the new SSE-enabled wait-for-hydra action
  • Points pr-validate.yml at input-output-hk/actions/wait-for-hydra@wait-for-hydra/sse-support
  • Uses hydra-status-url: https://ci.zw3rk.com for real-time build status via SSE

Do not merge — test PR only.

Switch pr-validate workflow to use the SSE-enabled wait-for-hydra
action branch. This connects to ci.zw3rk.com's SSE endpoint for
instant build status updates instead of polling the GitHub API.

Test PR — do not merge.
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Test-only PR to validate an SSE-enabled version of the wait-for-hydra GitHub Action by pointing PR validation at a non-default action ref and Hydra status endpoint.

Changes:

  • Switch pr-validate.yml to use input-output-hk/actions/wait-for-hydra@wait-for-hydra/sse-support.
  • Configure hydra-status-url: https://ci.zw3rk.com for the action.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/pr-validate.yml Outdated
runs-on: ubuntu-latest
steps:
- uses: input-output-hk/actions/wait-for-hydra@latest
- uses: input-output-hk/actions/wait-for-hydra@wait-for-hydra/sse-support
Copy link

Copilot AI Feb 28, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The workflow now references a feature/test branch of the external input-output-hk/actions/wait-for-hydra action (@wait-for-hydra/sse-support). If this change is ever merged, it will track a moving branch (and could break if the branch is force-pushed or deleted). Prefer pinning to an immutable ref (tag/release) or a commit SHA for stability.

Suggested change
- uses: input-output-hk/actions/wait-for-hydra@wait-for-hydra/sse-support
- uses: input-output-hk/actions/wait-for-hydra@v1

Copilot uses AI. Check for mistakes.
- uses: input-output-hk/actions/wait-for-hydra@wait-for-hydra/sse-support
with:
check: required
hydra-status-url: https://ci.zw3rk.com
Copy link

Copilot AI Feb 28, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Setting hydra-status-url to https://ci.zw3rk.com changes which Hydra instance PR validation waits on, and it differs from .github/workflows/main.yml (which relies on the action default). If this is intended only for this test PR, consider reverting before merge; otherwise, align the endpoint across workflows or make it configurable to avoid inconsistent CI behavior.

Suggested change
hydra-status-url: https://ci.zw3rk.com

Copilot uses AI. Check for mistakes.
The wait-for-hydra/sse-support branch now has the pipe subshell fix
(process substitution instead of pipe) so SSE exit codes propagate
correctly.
The wait-for-hydra SSE mode now uses read -t 60 for periodic timeout
checks and re-checks the one-shot endpoint when the SSE stream ends.
SSE through Cloudflare buffers events. Now the script polls the
bridge's one-shot endpoint every 120s as a safety net alongside
the SSE stream.
Cloudflare now bypasses cache for /status/* paths, enabling real-time
SSE event delivery.
@angerman
Copy link
Copy Markdown
Collaborator Author

SSE support verified end-to-end: wait-for-hydra@latest with SSE detected the required check, all 8 closure validations passed. Closing test PR.

@angerman angerman closed this Feb 28, 2026
angerman added a commit that referenced this pull request Feb 28, 2026
Use the hydra-github-bridge SSE endpoint for real-time build status
detection instead of polling the GitHub API. Reduces wait time from
minutes (poll interval) to seconds (event-driven).

Tested end-to-end in PR #241 — all validation jobs passed.
github-merge-queue Bot pushed a commit that referenced this pull request Feb 28, 2026
Use the hydra-github-bridge SSE endpoint for real-time build status
detection instead of polling the GitHub API. Reduces wait time from
minutes (poll interval) to seconds (event-driven).

Tested end-to-end in PR #241 — all validation jobs passed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants